Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Put the statistics code after http callback #2687

Merged
merged 1 commit into from
Oct 22, 2021
Merged

Put the statistics code after http callback #2687

merged 1 commit into from
Oct 22, 2021

Conversation

duiniuluantanqin
Copy link
Member

@duiniuluantanqin duiniuluantanqin commented Oct 19, 2021 •
edited by winlinvip
Loading

#1444

Phenomenon
RTMP process: Due to the statistics on_client being placed before the authentication module, when authentication fails, the cleanup statistics process will not be triggered, resulting in residual information.

TRANS_BY_GPT3

@duiniuluantanqin duiniuluantanqin linked an issue Oct 19, 2021 that may be closed by this pull request
On Publish error doesn't disconnect the client #1444
Closed
@winlinvip winlinvip merged commit a86f390 into ossrs:develop Oct 22, 2021
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xiaozhihong xiaozhihong xiaozhihong approved these changes

@winlinvip winlinvip Awaiting requested review from winlinvip

Assignees
No one assigned
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

On Publish error doesn't disconnect the client
3 participants
@duiniuluantanqin @xiaozhihong @winlinvip